Skip to content

feat: add param tests to test app#1023

Open
closetcaiman wants to merge 2 commits intomainfrom
feat/test-app-param-tests
Open

feat: add param tests to test app#1023
closetcaiman wants to merge 2 commits intomainfrom
feat/test-app-param-tests

Conversation

@closetcaiman
Copy link
Copy Markdown
Contributor

@closetcaiman closetcaiman commented Apr 15, 2026

Closes #

⚠️ Breaking changes ⚠️

Introduced changes

  • added param tests to test app
  • fixed broken tests for:
    • streamer
    • recorder

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web
  • Updated old arch android spec file

@closetcaiman closetcaiman changed the title feat: add param tests feat: add param tests to test app Apr 15, 2026
Copy link
Copy Markdown
Collaborator

@mdydek mdydek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make this test more verbose, add maybe artificial pauses, these tests should also provide visual explanation what is currently going on, but this implementation displays something for a fraction of a second and at the end it displays "scheduling exponential..."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants